Skip to content

Manage Cockpit process instance variables#78

Open
hauptmedia wants to merge 1 commit into
mainfrom
codex/cockpit-instance-variable-actions
Open

Manage Cockpit process instance variables#78
hauptmedia wants to merge 1 commit into
mainfrom
codex/cockpit-instance-variable-actions

Conversation

@hauptmedia

Copy link
Copy Markdown

Summary

  • add REST bindings and state for setting and deleting live process instance variables
  • add Add/Edit/Delete controls to the live process instance variables tab while keeping history variables read-only
  • preserve basic variable type parsing for booleans and numbers and add localized labels plus coverage
  • clean up three existing lint issues in the touched Processes page

Reference

Verification

  • node -e "JSON.parse(require('fs').readFileSync('public/locales/en-US/translation.json','utf8')); JSON.parse(require('fs').readFileSync('public/locales/de-DE/translation.json','utf8')); console.log('locale JSON ok')"
  • npx eslint src/api/resources/process_instance.js src/api/resources/process_instance.test.js src/pages/Processes.jsx src/pages/Processes.test.jsx
  • npx vitest run src/api/resources/process_instance.test.js src/pages/Processes.test.jsx
  • npx vitest run --maxWorkers=4
  • npm run build
  • browser smoke at http://127.0.0.1:5175/processes/proc:1/instances/inst-9999/vars: reached login boundary with title Operaton and no console errors

@sonarqubecloud

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant